Skip to content

Student auto feed script update #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Sep 9, 2019
Merged

Student auto feed script update #18

merged 6 commits into from
Sep 9, 2019

Conversation

pbailie
Copy link
Contributor

@pbailie pbailie commented Sep 7, 2019

Updates the following:

  • No longer sends an entire course to the NULL section when that course is not represented in the data feed (manual flag no longer required).
  • Mapped courses is no longer mutually exclusive with registered courses. This will allow a course's enrollment to be duplicated to another course. (i.e. csci1100 -> csci1199)
  • Some code cleanup/updates involving CSV processing.
  • SSL required in DB connection (was "preferred").
  • Removed code that is not used.
    • Removed remote CSV access. SSH2 bindings has proved to be very problematic (I can get a C assertion from libssh2!), but I will not support any form of network access to the student data file without encryption due to FERPA concerns. Meanwhile, we do not access the CSV over the network at RPI leaving little point in supporting this feature.
    • -g CLI argument removed. Legacy feature to auto-determine term code hasn't been used in a years, and I do not expect to ever use it again.

@pbailie
Update submitty_student_auto_feed.php
1282e2e 
* Courses that are registered in Submitty but have no enrollment data in the CSV will no longer have students sent to the NULL section.
* Now am mapped course may also be a fully registered course.  Useful to auto-enroll students into an optional psuedo-course of additional lessons.
@pbailie
Updates
d8051d0 
Some code maintenance.

I am no longer going to support remote CSV file access.  The SSH2 library is buggy and unreliable.  Further, it is not used at RPI.  This code has been removed.
@pbailie
Update
4d7f9f3 
WIP
@pbailie
Update submitty_student_auto_feed.php
a99efb4 
WIP
@pbailie
Update submitty_student_auto_feed.php
e2cabc2 
PR candidate
@pbailie
Update submitty_student_auto_feed.php
592e743 
Require SSL connection to DB.
pbailie added a commit to Submitty/submitty.github.io that referenced this pull request Sep 9, 2019
@pbailie
Update student_auto_feed.md
c532d29 
Update student auto feed documentation as related to PR Submitty/SysadminTools#18
@pbailie pbailie mentioned this pull request Sep 9, 2019
Merged
@bmcutler bmcutler merged commit 94f5a80 into master Sep 9, 2019
@bmcutler bmcutler deleted the auto_feed_script_update branch September 9, 2019 21:38
MasterOdin pushed a commit to Submitty/submitty.github.io that referenced this pull request Sep 10, 2019
@pbailie @MasterOdin
[Documentation] Update docs for latest version of student auto feed (#…
08b7d5c 
…164)

Update student auto feed documentation as related to PR Submitty/SysadminTools#18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pbailie @bmcutler